Skip to content

Conversation

@oleflb
Copy link
Contributor

@oleflb oleflb commented Jul 20, 2025

Why? What?

Use lowpass filter for all head movements.

Fixes #2052

ToDo / Known Issues

  • Remove velocity limit code

Ideas for Next Iterations (Not This PR)

If there are some improvements that could be done in a next iteration, describe them here.

How to Test

Describe how to test your changes. (For the reviewer)

@github-project-automation github-project-automation bot moved this to Request for Review in Development Jul 20, 2025
pub outer_minimum_pitch: f32,
pub outer_yaw: f32,
pub injected_head_joints: Option<HeadJoints<f32>>,
pub filter_smoothing_factor: f32,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
pub filter_smoothing_factor: f32,
pub smoothing_factor: f32,

let positions = request.positions;
let last_positions = self.last_request.positions;

let velocity_limited_positions = HeadJoints {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove velocity limiting

@schluis
Copy link
Contributor

schluis commented Jul 20, 2025

Hammerding 🥵

@schluis schluis self-assigned this Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Request for Review

Development

Successfully merging this pull request may close these issues.

Bottom Camera Destabilizes Walking

2 participants